-
Notifications
You must be signed in to change notification settings - Fork 30
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Shrinkdesc 92 #53
base: master
Are you sure you want to change the base?
Shrinkdesc 92 #53
Conversation
Hi Andrew and Jesper The cli maven plugin is running. The plugin generates the descriptors according the passed parameters. You can execute the transformation like this example: org.jboss.shrinkwrap.descriptors shrinkwrap-descriptors-cli-plugin ${project.version} transform install transform ${project.build.directory}/../../cli/src/test/resources/xml/schemasIronJacamar.xml ${project.build.directory}/../../cli-test/src/main/java ${project.build.directory}/../../cli-test/src/test/java ${project.build.directory}/cli trueNote, the can be empty, then no test classes are generated. Not done so far are:
Let me know how to proceed. Regards, |
…ector_1_0.dtd as converted xsd
…ector_1_0.dtd as converted xsd
…n module; service files are created optionally
…d are .gitignored
…iptors-metadata-parser, which it does not understand
Hi Andrew If Bartosz looks at the source code, then it is better to commit everything. So, the latest version includes the following features:
Some general comments about the plugin. The plugin will support much more descriptor out of the box because there are no hacks any more implemented covering a particular descriptor definition. I have to write little more test directly testing XSD specifications but this comes later. Have fun! Ralf |
Hi Andrew and Jesper
I changed the cli based on your suggestions. You can have a look.
@jesper, I am not so familiar with the mvn antrun plugin but I added a class called 'TransformerTask' which is similar to the ReportTask you have mentioned. I hope you can reuse this class via ant.
What is not covered so far are:
Let me know what has to be improved on this version.
Cheers,
Ralf